]> dgit.raspbian.org Git - gnome-shell.git/commitdiff
appDisplay: Fix folder loop in `_ensureDefaultFolders`
authorAdam Williamson <awilliam@redhat.com>
Thu, 10 Mar 2022 00:12:36 +0000 (16:12 -0800)
committerMarge Bot <marge-bot@gnome.org>
Sat, 12 Mar 2022 09:32:56 +0000 (09:32 +0000)
In testing on Fedora, gnome-shell crashes here:

JS ERROR: TypeError: DEFAULT_FOLDERS[folder] is undefined

This needs to be a "for of" loop, not a "for in" loop, because
`folders` is an array of the hash's keys, not the hash itself.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2238>

js/ui/appDisplay.js

index d8d0ff4f76795f339c37616b5ff2e04184aed070..821bfe3b8d41edaed446e0f573cf82f429e2d7cf 100644 (file)
@@ -1467,7 +1467,7 @@ class AppDisplay extends BaseAppView {
         this._folderSettings.set_strv('folder-children', folders);
 
         const { path } = this._folderSettings;
-        for (const folder in folders) {
+        for (const folder of folders) {
             const { name, categories, apps } = DEFAULT_FOLDERS[folder];
             const child = new Gio.Settings({
                 schema_id: 'org.gnome.desktop.app-folders.folder',